-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Send email docs #2432
feat: Send email docs #2432
Conversation
@@ -128,33 +128,6 @@ export const fetchPaymentRequestViaProxy = fetchPaymentViaProxyWithCallback( | |||
}, | |||
); | |||
|
|||
export const addGovPayPaymentIdToPaymentRequest = async ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I needed to move this into a shared utils.ts
as I was hitting an issue with circular dependencies on the build application.
c668dba
to
1ffa101
Compare
1ffa101
to
3e39070
Compare
Removed vultr server and associated DNS entries |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good & pizza working as expected 👍
What does this PR do?
/send-email
endpoints, refactor into module structurenotify
functionality to/lib
Regression tests passing on branch ✅ https://github.com/theopensystemslab/planx-new/actions/runs/6882410080